[SDK-448] Upgrade RN to the latest 0.79#860
Conversation
… Xcode 26.4 compatibility
…package.json and yarn.lock
|
|
Coverage Impact This PR will not change total coverage. 🚦 See full report on Qlty Cloud »🛟 Help
|
There was a problem hiding this comment.
Pull request overview
Updates the library workspace and example app from React Native 0.79.3 to 0.79.7, along with the associated React Native toolchain packages and regenerated iOS project metadata needed for the example app.
Changes:
- Bumps
react-nativeand related@react-native/*packages from 0.79.3 to 0.79.7 in the root workspace and example app. - Regenerates
yarn.lockand the example iOS Xcode project to align with the upgraded toolchain. - Adds an iOS CocoaPods post-install workaround for the
fmtpod in the example app.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| yarn.lock | Lockfile refresh for the RN/toolchain upgrade, plus an added direct baseline-browser-mapping entry. |
| package.json | Root workspace devDependency updates for RN 0.79.7 and related tooling. |
| example/package.json | Example app dependency/tooling updates to RN 0.79.7. |
| example/ios/ReactNativeSdkExample.xcodeproj/project.pbxproj | Regenerated Xcode project references and build phase IDs/scripts for the example app. |
| example/ios/Podfile | Adds a post-install CocoaPods workaround for the fmt target. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "@types/react": "^19.0.0", | ||
| "@typescript-eslint/eslint-plugin": "^8.13.0", | ||
| "@typescript-eslint/parser": "^8.13.0", | ||
| "baseline-browser-mapping": "latest", |
joaodordio
left a comment
joaodordio
left a comment
There was a problem hiding this comment.
All good apart from that baseline-browser-mapping latest. If I'm missing the point with this addition let me know 🙏
sumeruchat
left a comment
sumeruchat
left a comment
There was a problem hiding this comment.
LGTM — RN 0.79.7 bump looks safe, local verification passed (install, pods, typecheck, lint, test, build, example iOS on Xcode 26.4.1). Trusting CI for Android.
One non-blocking nit before merge: baseline-browser-mapping was added as a direct devDependency pinned to latest, but it isn't imported anywhere in the repo and is already pulled in transitively via browserslist. Pinning a top-level dep to a moving npm tag in a dependency-safety PR is a bit risky — consider either:
- dropping the direct dep and relying on the transitive refresh in
yarn.lock, or - pinning it to an exact version if it does need to be top-level.
Also worth flagging in release notes: the example/ios/Podfile fmt C++17 override is example-only and won't ship to consumers — if customers on RN 0.79.x + Xcode 26.4 hit the same fmt build error, they'll need an app-level Podfile workaround. Might be worth a quick docs/support note.
… in package.json and yarn.lock
🔹 JIRA Ticket(s) if any
✏️ Description